Skip to content

Conversation

@indigo423
Copy link
Member

@indigo423 indigo423 commented Jan 15, 2026

Add installation instructions to support CentOS 10 based Linux distributions.

Reviewer Hints

The OpenJDK 17 isn't available in RPM repositories and we need to add Eclipse Temurin repositories to get a maintained OpenJDK 17 installed.

The additions here will be a good foundation for NMS-19455.

I have rebased the branch to resolve the conflicts and minimise the churn for merging the PR.

External References

@github-actions github-actions bot added the docs label Jan 15, 2026
@mfuhrmann mfuhrmann self-requested a review January 15, 2026 20:31
@mfuhrmann mfuhrmann marked this pull request as ready for review January 15, 2026 20:31
mfuhrmann
mfuhrmann previously approved these changes Jan 15, 2026
@indigo423 indigo423 changed the title Jira/nms 19060 NMS-19060: docs(install): Add installation instructions for CentOS 10 Jan 15, 2026
@indigo423 indigo423 changed the title NMS-19060: docs(install): Add installation instructions for CentOS 10 NMS-19060: Add installation instructions for CentOS 10 Jan 15, 2026
@indigo423 indigo423 force-pushed the jira/NMS-19060 branch 4 times, most recently from d6b759c to 775892c Compare January 16, 2026 12:13
@indigo423 indigo423 force-pushed the jira/NMS-19060 branch 3 times, most recently from 80c2c64 to fcdfa68 Compare January 23, 2026 00:23
* Fix includes and removed duplicate include for CentOS 9 and 10.
* Simplify variables for compatible versions used in the system tp
requirements.
* Fixed build warnings for Kafka topic placeholders and masquarded them
  so that they are not tried to be resolved as an Antora variable.
* Remove Rocky Linux and just keep it to the major upstream RPM/DEB
  distros

Signed-off-by: Ronny Trommer <ronny@no42.org>
dino2gnt
dino2gnt previously approved these changes Jan 23, 2026
Signed-off-by: Ronny Trommer <ronny@no42.org>
Signed-off-by: Ronny Trommer <ronny@no42.org>
@indigo423
Copy link
Member Author

indigo423 commented Jan 23, 2026

@dino2gnt I have tested this with RHEL 10, I have removed the need to edit the opennms-datasources.xml and also to the installation verification using the tree command.

Because we differentiate between CentOS/RHEL and Debian/Ubuntu, I've replaced $OPENNMS_HOME with /opt/opennms and /usr/share/opennms to make these commands copy & paste friendly.

@indigo423 indigo423 requested a review from dino2gnt January 23, 2026 13:59
@dino2gnt
Copy link
Contributor

I've replaced $OPENNMS_HOME with /opt/opennms and /usr/share/opennms to make these commands copy & paste friendly.

$OPENNMS_HOME should be c&p friendly, the packages add it to profile.d.

Remember that all the paths for Debuntu changed to match the RPMs in 35.

@indigo423
Copy link
Member Author

indigo423 commented Jan 23, 2026

I've replaced $OPENNMS_HOME with /opt/opennms and /usr/share/opennms to make these commands copy & paste friendly.

$OPENNMS_HOME should be c&p friendly, the packages add it to profile.d.

Remember that all the paths for Debuntu changed to match the RPMs in 35.

yes, we need to clean this up in a PR against release-35.x. With /opt/opennms we can simplify a lot here as well.

The $OPENNMS_HOME wasn't set for the user doing the installation.

@dino2gnt
Copy link
Contributor

The $OPENNMS_HOME wasn't set for the user doing the installation.

If the file isn't being created on EL10, we should determine why.

@indigo423
Copy link
Member Author

indigo423 commented Jan 23, 2026

The $OPENNMS_HOME wasn't set for the user doing the installation.

If the file isn't being created on EL10, we should determine why.

It's pretty simple, profile.d is executed when you do a login. When you follow these instructions you run in two issues, a) the instruction don't tell you to logout/login again or source your bash or whatever you use after you installed the opennms package and b) it's not transparent that we install something in profile.d affecting the users shell.

With 35.x forward we can get rid of that stuff anyway and I think we should. It's just error-prone.

@dino2gnt
Copy link
Contributor

We don't source it in %post ? We should :D

I'm fine with whatever. I trust that y'all are pedantic enough to get working installation instructions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants